modifications to dart.h and vterbi.cpp files - #4
Conversation
|
Thanks for this! I see this unmerged PR in the upstream MeCab repo from a year ago taku910/mecab#66 so I'm happy to accept the fix to get this to compile as C++17. In order to keep the differences with the upstream/original C++ code as minimal as possible though, can I beg you to revert your clang-format changes? Let me know, if you're busy I can try to undo the whitespace changes, keeping just the Again, thank you! |
|
Aiite will revert. |
|
Hi will work on this. been very busy with work |
|
I just fixed the clangtidy formatted files |
|
Thanks! Could you help me understand the rewrite of Also, I see that your taku910/mecab#66 has a really small diff, do you think I could just merge that patch into this repo? |
oh so sorry about that, its a mix up from my end. will resolve that asap |
The C++ storage type specifier "Register" was removed to enable build for C++17 also the edit files were clang formatted